Skip to content

Harden YouTube section workflow script execution in youtube-section-update.yml#249

Merged
DavidKRK merged 2 commits into
gh-pagesfrom
copilot/youtube-section-update
May 29, 2026
Merged

Harden YouTube section workflow script execution in youtube-section-update.yml#249
DavidKRK merged 2 commits into
gh-pagesfrom
copilot/youtube-section-update

Conversation

Copilot AI commented May 29, 2026

Copy link
Copy Markdown
Contributor

The YouTube section update workflow was failing at parse/runtime boundaries, reported as a YAML syntax error around the inline script block. This change makes script execution deterministic by removing shell/YAML ambiguity in that step.

  • Workflow execution model

    • Switched the script step from shell: python to shell: bash.
    • Runs Python explicitly through a heredoc, so GitHub Actions parses YAML cleanly and Python receives the script exactly as authored.
  • Behavior preserved

    • Kept the existing content-generation logic intact for:
      • music.html YouTube auto-section injection
      • assets/css/style.css conditional YouTube CSS append
  • Relevant change

    - name: Injecter les videos et le CSS via Python
      shell: bash
      run: |
        python <<'PY'
        import json
        import re
        import os
        # existing logic unchanged...
        PY

@bolt-new-by-stackblitz

Copy link
Copy Markdown

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@DavidKRK DavidKRK marked this pull request as ready for review May 29, 2026 04:49
Copilot AI review requested due to automatic review settings May 29, 2026 04:49

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Switches the YouTube section update workflow from shell: python to shell: bash running Python via a quoted heredoc, to avoid YAML/shell parsing ambiguity while preserving the script's behavior.

Changes:

  • Change shell: pythonshell: bash for the inject step.
  • Wrap the existing Python logic inside python <<'PY' ... PY heredoc.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@DavidKRK DavidKRK merged commit 7930d9e into gh-pages May 29, 2026
10 checks passed
@DavidKRK

Copy link
Copy Markdown
Owner

@copilot je peux fermer la branche ?

@DavidKRK

Copy link
Copy Markdown
Owner

Pull request overview

Switches the YouTube section update workflow from shell: python to shell: bash running Python via a quoted heredoc, to avoid YAML/shell parsing ambiguity while preserving the script's behavior.

Changes:

  • Change shell: pythonshell: bash for the inject step.
  • Wrap the existing Python logic inside python <<'PY' ... PY heredoc.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Je peux fermer la branche ?

@DavidKRK DavidKRK deleted the copilot/youtube-section-update branch May 30, 2026 22:47
@DavidKRK DavidKRK requested a review from Copilot May 30, 2026 23:41

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants